Switch Jenkins image tracking to registry tag due to Samples Operator… … deprecation #1151
Conversation
ravanelli
changed the title
ravanelli
force-pushed
the
pr/fix_stg
branch
2 times, most recently
from
2ea2426 to
6af98f5
Compare
ravanelli
force-pushed
the
pr/fix_stg
branch
2 times, most recently
from
41d0ed5 to
6cb26c3
Compare
| kind: ImageStream | ||
| metadata: | ||
| name: jenkins |
There was a problem hiding this comment.
Now we have two imagestreams named jenkins. The other one is on line 50 of jenkins-s2i.yaml. I think one of them has to go.
There was a problem hiding this comment.
The 2 are different, one defines the buildconfig to add our plugins and so on, and the other basically defines the DeploymentConfig. I think make sense to keep it as it is.
There was a problem hiding this comment.
I'm pretty sure we can't define two image streams in the same namespace with the same name :).
Previously we had the one in jenkins-s2i.yaml because we needed the output from our buildconfig to go somewhere and a jenkins imagestream in our own namespace was where we wanted it to go, so we had to define a jenkins imagestream in our own namespace.
Now we are already defining a jenkins imagestream in our own namespace here in jenkins-images.yaml (previously we were pulling from the jenkins imagestream in the openshift namespace).
So.. we are already defining the jenkins imagestream here in our namespace, we don't need to define it again. The buildconfigs will just output to this imagestream on the :withcert or :latest tags.
There was a problem hiding this comment.
aaa never mind, I got it wrong. I was thinking about jenkins.yaml and jenkins-s2i.yaml. Got it now, thanks for the exaplanation
ravanelli
force-pushed
the
pr/fix_stg
branch
2 times, most recently
from
5cfc40b to
54b9c11
Compare
manifests/jenkins-with-cert.yaml
Outdated
| RUN if grep 'dummy' /etc/pki/ca-trust/source/anchors/root-ca.crt; then \ | ||
| rm /etc/pki/ca-trust/source/anchors/root-ca.crt; \ | ||
| else \ | ||
| update-ca-trust; \ | ||
| fi |
There was a problem hiding this comment.
Did you try the EOF ? I know it works in newer podman build, but maybe OpenShift builds it doesn't work yet?
RUN <<EOF
if
else
fi
EOF
There was a problem hiding this comment.
Yes, it doesn't seem to work
… deprecation Starting with OpenShift 4.13, the Cluster Samples Operator has been downsized and no longer provides updates for non-S2I images like Jenkins. The `latest` tracked tag was pointing to an image that hadn't been updated in over two years. This commit updates the image reference to follow the specific registry tag directly (registry.redhat.io/ocp-tools-4/jenkins-rhel9:v4.17.0), ensuring we get the latest maintained version going forward. To archive this we need to create our own ImageStreams for both the Jenkins base image and the Jenkins agent image, replacing the deprecated Samples Operator content. Signed-off-by: Renata Ravanelli <rravanel@redhat.com>
- Update documentation to reflect changes - Add note about how to work with the Fedora Staging pipecfg. Signed-off-by: Renata Ravanelli <rravanel@redhat.com>
Added jenkins-stg-agent-base.yaml to work around limitations in the Staging environment. Also updated documentation to note the unreliability of the Staging registry and instructed users to use their own agent image.